Skip to content

Comments

Copilot/update firewall access editors#17599

Merged
pelikhan merged 2 commits intomainfrom
copilot/update-firewall-access-editors
Feb 21, 2026
Merged

Copilot/update firewall access editors#17599
pelikhan merged 2 commits intomainfrom
copilot/update-firewall-access-editors

Conversation

@pelikhan
Copy link
Contributor

No description provided.

Copilot AI and others added 2 commits February 21, 2026 22:17
… weekly-editors-health-check

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI review requested due to automatic review settings February 21, 2026 22:34
@pelikhan pelikhan merged commit 82f7042 into main Feb 21, 2026
86 checks passed
@pelikhan pelikhan deleted the copilot/update-firewall-access-editors branch February 21, 2026 22:34
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates firewall access rules for the weekly editors health check workflow by replacing wildcard domain patterns with specific allowed domains. This change improves security by restricting network access to only the explicitly required domains.

Changes:

  • Replaced *.github.io and *.github.com wildcards with three specific domains: github.github.com, ashleywolf.github.io, and mossaka.github.io
  • Updated all firewall configurations (network allowlist, Playwright domains, command-line flags, and environment variables) to use the specific domains
  • Updated the workflow metadata hash to reflect the configuration changes

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/weekly-editors-health-check.md Updated network allowlist to replace wildcards with specific domains for the three editor sites
.github/workflows/weekly-editors-health-check.lock.yml Updated generated workflow file with specific domains in all firewall configurations (allowed_domains array, awf command flags, GH_AW_ALLOWED_DOMAINS env var, Playwright MCP server config) and updated metadata hash

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants